Integrate Application Insights and OpenTelemetry for CLI observability#1838
Merged
krusty93 merged 4 commits intoJun 26, 2026
Conversation
Contributor
Author
|
Few screenshots: Operation duration and status: Log events using logtape open telemetry sink ( Each span logs the dependencies: |
Contributor
|
Since the CLI is a public package available to anyone, any tracking mechanism must be opt-in and accompanied by a privacy policy. The same applies to pipelines. Alternatively, we must restrict tracking to internal usage only |
krusty93
force-pushed
the
CES-1615-tracciare-con-application-insights-lutilizzo-ed-eventuali-errori-della-dx-cli
branch
from
3164682 to
bffee13
Compare
Contributor
Author
restricted to pagopa users only, via github |
krusty93
force-pushed
the
CES-1615-tracciare-con-application-insights-lutilizzo-ed-eventuali-errori-della-dx-cli
branch
from
bffee13 to
85a4b82
Compare
krusty93
force-pushed
the
CES-1615-tracciare-con-application-insights-lutilizzo-ed-eventuali-errori-della-dx-cli
branch
2 times, most recently
from
ecff58e to
0d4b05f
Compare
krusty93
force-pushed
the
CES-1615-tracciare-con-application-insights-lutilizzo-ed-eventuali-errori-della-dx-cli
branch
from
0d4b05f to
fa45050
Compare
krusty93
force-pushed
the
CES-1615-tracciare-con-application-insights-lutilizzo-ed-eventuali-errori-della-dx-cli
branch
from
03201bb to
80b449d
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds end-to-end observability for the dx CLI by integrating Azure Monitor (Application Insights) and OpenTelemetry, including a single root span for each CLI invocation and log export via Logtape → OTel. It also tightens CLI preconditions (GitHub auth) and updates workspace dependency catalogs/lockfiles accordingly.
Changes:
- Add Azure Monitor/OpenTelemetry preload + runtime enablement, root span tracing, and OTel log sink wiring for the CLI.
- Enforce GitHub authentication for all CLI commands except
--help/--version, with telemetry enabled only forpagopaorg members. - Update dependency catalogs/lockfile and documentation to reflect new auth + telemetry behavior.
Reviewed changes
Copilot reviewed 15 out of 17 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| pnpm-workspace.yaml | Adds/updates OpenTelemetry-related catalog entries and normalizes quoting/format. |
| pnpm-lock.yaml | Updates lockfile to include new OTel/Logtape/Azure Monitor dependencies and related transitive changes. |
| package.json | Updates packageManager and adds pnpm overrides at root. |
| apps/website/docs/dx-cli/requirements.md | Documents mandatory GitHub auth and telemetry enablement rules. |
| apps/cli/src/index.ts | Adds early auth/telemetry gating, OTel log sink, root span lifecycle, and tool-version enrichment. |
| apps/cli/src/adapters/plop/actions/setup-pnpm.ts | Strips debugger-related env vars from spawned processes. |
| apps/cli/src/adapters/octokit/index.ts | Adds isPagopaOrgMember helper to gate telemetry. |
| apps/cli/src/adapters/octokit/tests/index.test.ts | Adds unit tests for isPagopaOrgMember. |
| apps/cli/src/adapters/commander/commands/doctor.ts | Avoids process.exit() to allow telemetry flush in finally. |
| apps/cli/src/adapters/commander/commands/add.ts | Extends action result shape to include the environment payload. |
| apps/cli/src/adapters/azure-monitor/telemetry.ts | Adds flushTelemetry() helper to shutdown/flush Azure Monitor SDK safely. |
| apps/cli/src/adapters/azure-monitor/instrumentation.ts | Adds preload module hook registration + Azure Monitor enablement and resource attribute setup. |
| apps/cli/README.md | Documents GitHub auth requirement and telemetry scope. |
| apps/cli/package.json | Adds Azure Monitor + OTel deps and Logtape OTel sink deps to the CLI app. |
| apps/cli/bin/index.js | Ensures instrumentation preload is imported before the CLI entrypoint via sequential dynamic imports. |
| .nx/version-plans/version-plan-1781270390103.md | Version plan for docs change (GitHub login requirement). |
| .nx/version-plans/version-plan-1781270291055.md | Version plan for CLI telemetry behavior change. |
Files not reviewed (1)
- pnpm-lock.yaml: Generated file
krusty93
force-pushed
the
CES-1615-tracciare-con-application-insights-lutilizzo-ed-eventuali-errori-della-dx-cli
branch
from
94ef95f to
6f50e56
Compare
krusty93
force-pushed
the
CES-1615-tracciare-con-application-insights-lutilizzo-ed-eventuali-errori-della-dx-cli
branch
from
6f50e56 to
1ab8c3b
Compare
krusty93
force-pushed
the
CES-1615-tracciare-con-application-insights-lutilizzo-ed-eventuali-errori-della-dx-cli
branch
from
1ab8c3b to
16ac30e
Compare
krusty93
force-pushed
the
CES-1615-tracciare-con-application-insights-lutilizzo-ed-eventuali-errori-della-dx-cli
branch
from
16ac30e to
b1cfcc6
Compare
mamu0
reviewed
Jun 16, 2026
krusty93
force-pushed
the
CES-1615-tracciare-con-application-insights-lutilizzo-ed-eventuali-errori-della-dx-cli
branch
from
b1cfcc6 to
c2de757
Compare
krusty93
force-pushed
the
CES-1615-tracciare-con-application-insights-lutilizzo-ed-eventuali-errori-della-dx-cli
branch
from
c2de757 to
ce583b9
Compare
krusty93
force-pushed
the
CES-1615-tracciare-con-application-insights-lutilizzo-ed-eventuali-errori-della-dx-cli
branch
from
ce583b9 to
91213a8
Compare
krusty93
force-pushed
the
CES-1615-tracciare-con-application-insights-lutilizzo-ed-eventuali-errori-della-dx-cli
branch
from
91213a8 to
4b7d615
Compare
krusty93
force-pushed
the
CES-1615-tracciare-con-application-insights-lutilizzo-ed-eventuali-errori-della-dx-cli
branch
from
7ae754a to
03f40ac
Compare
krusty93
force-pushed
the
CES-1615-tracciare-con-application-insights-lutilizzo-ed-eventuali-errori-della-dx-cli
branch
from
03f40ac to
aa73aec
Compare
Contributor
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://proud-mud-0405c6703-1838.westeurope.1.azurestaticapps.net |
mamu0
approved these changes
Jun 26, 2026
krusty93
deleted the
CES-1615-tracciare-con-application-insights-lutilizzo-ed-eventuali-errori-della-dx-cli
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Implement OpenTelemetry logging for improved observability, including root span tracing for CLI commands. Enhance telemetry capabilities by concurrently fetching tool versions and adding Azure Monitor service name.